-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
filter & chart header UI enhancements #283
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for submitting your first PR to this repository. We appreciate your contribution! Our team will review your code within 3 business days. We <3 open source and are so glad you do, too!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for contributing! It looks like the number next to the filter icon is the number of available filters. This isn't your fault, it wasn't specified, but the number should be the number of filters actually being used - regardless of how many there are. So when the page first loads, there should be no number, because no filters are being applied.
Also, on my install I'm seeing broken images for the 'down arrow' next to the filters as well as the dates:
Congratulations! 🎉 🙌 🎆 ㊗️ This PR is fantastic and has been merged into the code. Thanks so much for contributing! If you'd like, you can request to work on another issue. We really appreciate your effort! |
Hi @deveshchatuphale7! |
Sure! Thanks! |
Related Issue
Fixes #17
Checklist:
Optional items:
Optional: Add any explanations here
Modified CSS & HTML for filter-panel component. Also added classes to dashboard components from bootstrap & root.css
Optional: Add any relevant screenshots here